Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CURA-11084] Add missing 'perform_prime' member-variable. #1960

Merged
merged 1 commit into from
Sep 26, 2023

Conversation

rburema
Copy link
Member

@rburema rburema commented Sep 26, 2023

Note that this needs the updated GRPC definitions as well! -- This caused that variable to always be (re)set to false, even when no plugin was present, due to the 'identity' conversion taking place. (Maybe we should stop that in general, since that implies that we copy the data even when no plugin is present.) In any case, even if it wouldn't have caused this particular bug, it's an oversight that should be fixed anyway.

See also: Ultimaker/CuraEngine_grpc_definitions#13

Note that this needs the updated GRPC definitions as well! -- This caused that variable to always be (re)set to false, even when no plugin was present, due to the 'identity' conversion taking place. (Maybe we should stop that in general, since that implies that we copy the data even when no plugin is present.) In any case, even if it wouldn't have caused this particular bug, it's an oversight that should be fixed anyway.

should fix CURA-11084
@github-actions
Copy link
Contributor

Unit Test Results

0 tests   - 26   0 ✔️  - 26   0s ⏱️ -13s
0 suites  -   1   0 💤 ±  0 
0 files    -   1   0 ±  0 

Results for commit d78bb6d. ± Comparison against base commit c2139cf.

@casperlamboo casperlamboo merged commit 40cd391 into 5.5 Sep 26, 2023
16 of 19 checks passed
@rburema rburema deleted the CURA-11084_fix_missing_prime branch June 26, 2024 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants